Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow attributes to be added to anchor items in header, footer, breadcrumbs, tabs and error-summary components #993

Merged
merged 6 commits into from
Sep 13, 2018

Conversation

kr8n3r
Copy link

@kr8n3r kr8n3r commented Sep 10, 2018

This has been requested though support and in our own work on the Prototype Kit.

Allow attributes to be added with Nunjucks macros to:

  • breadcrumb links
  • header navigation links
  • tabs links and tabs panels
  • error-summary links
  • footer navigation and meta links

For each of the components the work includes

  • Enabling HTML attributes to be added in the component's Nunjucks template
  • Adding tests to check that attributes render correctly
  • Documenting new functionality in README.njk
  • Generating an updated README.md file

Trello ticket: https://trello.com/c/9BPK4u7M/414-1-allow-attributes-to-be-specified-on-items-using-macro

Fixes: #904

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-993 September 10, 2018 15:23 Inactive
@kr8n3r kr8n3r changed the title Allow attributes on child items for Allow attributes to be added to some child items on Header, Footer, Breadcrumbs, Tabs and Error-summary components Sep 10, 2018
@kr8n3r kr8n3r force-pushed the allow-attributes-on-child-items branch from f8405e9 to 6e07f5f Compare September 10, 2018 15:28
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-993 September 10, 2018 15:28 Inactive
@kr8n3r kr8n3r changed the title Allow attributes to be added to some child items on Header, Footer, Breadcrumbs, Tabs and Error-summary components Allow attributes to be added to some child items on header, footer, Breadcrumbs, tabs and error-summary components Sep 10, 2018
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-993 September 10, 2018 15:31 Inactive
@kr8n3r kr8n3r changed the title Allow attributes to be added to some child items on header, footer, Breadcrumbs, tabs and error-summary components Allow attributes to be added to some anchor items on header, footer, Breadcrumbs, tabs and error-summary components Sep 10, 2018
@kr8n3r kr8n3r changed the title Allow attributes to be added to some anchor items on header, footer, Breadcrumbs, tabs and error-summary components Allow attributes to be added to anchor items in header, footer, breadcrumbs, tabs and error-summary components Sep 10, 2018
@kr8n3r kr8n3r added this to the [NEXT] milestone Sep 11, 2018
igloosi added 2 commits September 11, 2018 11:58
- Allow HTML atrributes to be added to breadcrumb links
in the Nunjucks macro template
- Add tests for check that attributes render correctly
- Document new functionality in README.njk
- Generate an updated README.md file
- Allow HTML atrributes to be added to header navigation links
in the Nunjucks macro template
- Add tests to check that attributes render correctly
- Document new functionality in README.njk
- Generate an updated README.md file
@kr8n3r kr8n3r force-pushed the allow-attributes-on-child-items branch from 31aa3b6 to 49dac1f Compare September 11, 2018 10:59
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-993 September 11, 2018 10:59 Inactive
Copy link
Contributor

@NickColley NickColley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me 👍

src/components/tabs/README.md Outdated Show resolved Hide resolved
igloosi added 4 commits September 13, 2018 09:24
- Allow HTML atrributes to be added to tabs links and tabs panels
in the Nunjucks macro template
- Add tests to check that attributes render correctly
- Document new functionality in README.njk
- Generate an updated README.md file
- Allow HTML atrributes to be added to error-summary links
in the Nunjucks macro template
- Add tests for check that attributes render correctly
- Document new functionality in README.njk
- Generate an updated README.md file
    - Allow HTML atrributes to be added to footer navigation and meta links
 in the Nunjucks macro template
    - Add tests to check that attributes render correctly
    - Document new functionality in README.njk
    - Generate an updated README.md file
@kr8n3r kr8n3r force-pushed the allow-attributes-on-child-items branch from 49dac1f to ae9bfd2 Compare September 13, 2018 08:24
@kr8n3r kr8n3r merged commit d76277e into master Sep 13, 2018
@kr8n3r kr8n3r deleted the allow-attributes-on-child-items branch September 13, 2018 08:27
@kr8n3r kr8n3r mentioned this pull request Sep 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants